Skip to content

updated usage example of hygon assigning tasks to specific DCUs#87

Merged
Nimbus318 merged 1 commit intoProject-HAMi:masterfrom
peachest:master
Jul 21, 2025
Merged

updated usage example of hygon assigning tasks to specific DCUs#87
Nimbus318 merged 1 commit intoProject-HAMi:masterfrom
peachest:master

Conversation

@peachest
Copy link
Copy Markdown

@peachest peachest commented Jul 18, 2025

With this PullRequest fixed, the id of the hygon DCU should now be unique in the cluster, so you can use hygon.com/use-uuid annotation to specify the Pod scheduling range.

Sample documentation for assigning tasks to specific hygon DCUs has been added, taking into account existing documentation

@hami-robot
Copy link
Copy Markdown
Contributor

hami-robot bot commented Jul 18, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: peachest
Once this PR has been reviewed and has the lgtm label, please assign wawa0210 for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hami-robot hami-robot bot requested review from archlitchi and wawa0210 July 18, 2025 09:42
@netlify
Copy link
Copy Markdown

netlify bot commented Jul 18, 2025 •
edited
Loading

✅ Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 7c566a1
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/687db8be6f75d500083702c4
😎 Deploy Preview https://deploy-preview-87--project-hami.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@hami-robot
Copy link
Copy Markdown
Contributor

hami-robot bot commented Jul 18, 2025

Welcome @peachest! It looks like this is your first PR to Project-HAMi/website 🎉

@hami-robot hami-robot bot added the size/M label Jul 18, 2025
@peachest peachest mentioned this pull request Jul 18, 2025
Merged
@Nimbus318
Copy link
Copy Markdown
Collaborator

Nimbus318 commented Jul 19, 2025

@peachest I forgot to mention earlier — please also update the sidebar config accordingly to include the two new files you added.

website/sidebars.js

Lines 96 to 112 in 702238b

{
"type": "category",
"label": "Share Hygon DCU devices",
"items": [
"userguide/Hygon-device/enable-hygon-dcu-sharing",
"userguide/Hygon-device/specify-device-memory-usage",
"userguide/Hygon-device/specify-device-core-usage",
{
"type": "category",
"label": "Examples",
"items": [
"userguide/Hygon-device/examples/allocate-core-and-memory",
"userguide/Hygon-device/examples/allocate-exclusive"
]
}
]
},

You’ll need to add:

  • "userguide/Hygon-device/specify-device-uuid-to-use" at the top level of the items array (alongside specify-device-memory-usage, etc.)
  • "userguide/Hygon-device/examples/specify-certain-cards" inside the "Examples" subcategory.

This ensures they show up properly in the documentation sidebar. Thanks again!

updated usage example of hygon assigning tasks to specific DCUs
7c566a1 
Signed-off-by: houyuxi <yuxi.hou@transwarp.io>
@Nimbus318
Copy link
Copy Markdown
Collaborator

Nimbus318 commented Jul 21, 2025

/lgtm

@hami-robot hami-robot bot added the lgtm label Jul 21, 2025
@Nimbus318
Copy link
Copy Markdown
Collaborator

Nimbus318 commented Jul 21, 2025

/approved

@Nimbus318 Nimbus318 merged commit 401d7b4 into Project-HAMi:master Jul 21, 2025
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@archlitchi archlitchi Awaiting requested review from archlitchi

@wawa0210 wawa0210 Awaiting requested review from wawa0210

Assignees

@Nimbus318 Nimbus318

Labels

dco-signoff: yes lgtm size/M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@peachest @Nimbus318